Skip to content

Conversation

eduardoboucas
Copy link
Member

Summary

We're referencing a deploy that has been deleted.

@eduardoboucas eduardoboucas requested a review from a team as a code owner August 7, 2025 10:27
@eduardoboucas
Copy link
Member Author

Superseded by #6609 (review).

@eduardoboucas eduardoboucas reopened this Aug 7, 2025
@JakeChampion JakeChampion enabled auto-merge (squash) August 7, 2025 10:33
@@ -1,5 +1,5 @@
// this needs to be updated whenever there's a change to globalThis.Netlify in bootstrap
import { Netlify } from "https://64e8753eae24930008fac6d9--edge.netlify.app/bootstrap/index-combined.ts"
import { Netlify } from "https://68945e2afae503000862a504--edge.netlify.app/bootstrap/index-combined.ts"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not ideal, but should we use getURL helper from edge-functions-bootstrap package to get bootstrap url? Afaik it provides branch urls (derived from package version name) and last deploys on branch deploys should never get deleted given our current soft deletion logic?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh nvm just noticed this change is superseded by #6609 (review).

@eduardoboucas
Copy link
Member Author

Closing (again) in favour of #6609!

auto-merge was automatically disabled August 7, 2025 12:32

Pull request was closed

@mrstork mrstork deleted the fix/bootstrap-version branch August 7, 2025 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants